Skip to content

[T7][T14-C4]#72

Open
SukiTsang wants to merge 670 commits intonus-cs2103-AY1617S1:masterfrom
CS2103AUG2016-T14-C4:master
Open

[T7][T14-C4]#72
SukiTsang wants to merge 670 commits intonus-cs2103-AY1617S1:masterfrom
CS2103AUG2016-T14-C4:master

Conversation

@SukiTsang
Copy link

No description provided.

@hang15
Copy link

hang15 commented Oct 10, 2016

Some comments on your user guide:

Features/User experience

  1. Your datetime input seems to be very rigid or fixed. Try to accommodate/allow some degree of flexibility, eg. DD-MM-YY. What about an event with a fixed date and no fixed time?
  2. There are inconsistencies in your user guide on the datetime format. Please indicate if it is meant to be flexible.
  3. You may want to indicate/think through in detail how your undo feature should work. For eg. how many undos are allowed?

Housekeeping -
You need to acknowledge addressbook-level4 or its parent project/org for the code base.

Skaty and others added 27 commits November 3, 2016 10:16
Fix documentation and personal TA code issues
Just a simple fix for my code. No tests required, since it is a docstring.
Allow TaskList to be filtered through preset filters
Allow user to see the actual open and close time of tasks
Add link to DeveloperGuide and update mock-up picture on README
Skaty and others added 30 commits November 7, 2016 21:40
Remove open and close time tests as they are non-deterministic
Update with new commands in help page
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants